Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add new architecture support #2033

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

alanjhughes
Copy link

@alanjhughes alanjhughes commented Mar 7, 2024

Adds support for the new architecture with bridgeless enabled which will be the default in 0.74

Tested on

  • OId Arch ✅
  • New Arch
    • Bridgeless enabled
      • iOS ✅
      • Android ✅
    • Bridgeless disabled ✅
      • iOS ✅
      • Android ✅

Note: On Android, onActivityResult was not implemented in RN 0.74 RC2 on bridgeless. It will be added in RC3. I have already tested against this change and it works as expected
Issues have been resolved in RC3 and now everything works correctly

@brentvatne
Copy link

looking forward to this landing and getting us one step closer to new arch support in the ecosystem! @ivpusic any chance you could merge this and get it into a new release, so that we can include it in a list of libraries supported in the new arch for react-native 74?

@brentvatne
Copy link

@ivpusic - any chance you can land this?

@eeston
Copy link

eeston commented Oct 25, 2024

Is this PR still mergable or is it stale?

@dcenatiempo
Copy link

dcenatiempo commented Nov 4, 2024

@ivpusic

Can this please get merged?

Preventing us from updating to RN 76

@Maker-Mark
Copy link

@ivpusic Any ETA on getting this merged?

@jsheffers
Copy link

Would love this!

@Arjit0762
Copy link

Please merge this one.. Needed

@SolankiYogesh
Copy link

@alanjhughes i tested your patch but its not working for me

image

@ubaidkolad
Copy link

Please get this merged, blocked to update to RN 0.76

@breezertwo
Copy link

Please get this merged. Only package preventing us from updating:
Unsupported on New Architecture: react-native-image-crop-picker

@brentvatne
Copy link

@breezertwo - in the meantime, you could consider migrating to expo-image-picker if the features there meet your requirements. at expo we opened this pr to help this library back in may but it may just be that the maintainer doesn't have time to dedicate to it anymore.

@ivpusic
Copy link
Owner

ivpusic commented Jan 15, 2025

I can confirm that I am currently short on time. @brentvatne have you done some testing on your side for this PR, or somebody else? does everything seems to be fine with the ios and android?

@alanjhughes could you take a look into merge conflict?

@alanjhughes
Copy link
Author

@ivpusic the PR is old now so I'll look at bringing it up to date

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.